Description & motivation 💭

PR #3125 (ESLint v9 upgrade) added resolve() calls to many routeFor* functions. Functions that compose other routeFor* functions ended up calling resolve() multiple times resulting in the base path being prepended multiple times. For example, routeForWorkflows() called resolve(routeForNamespace(...)) which double-resolved to /{base_path)/{base_path)/namespaces/default/workflows.

This PR fixes this by:

• Fixing instances where routeFor* functions that compose other routeFor* functions were calling resolve() again.
• Adding gotoResolved() utility that creates a type-safe wrapper around goto() that accepts only ResolvedRoute values, centralizing the eslint-disable for svelte/no-navigation-without-resolve in one place.
• Adding branded ResolvedRoute type to replace SvelteKit's ResolvedPathname (which is just string) with a branded type that TypeScript enforces at compile time, preventing raw strings from being passed where a resolved route is expected in gotoResolved().
• Adding base path regression tests and verifying every exported routeFor* functions includes the base path
  exactly once. An additional test automatically fails if a new routeFor* function is added without a corresponding test case.

Screenshots (if applicable) 📸

Design Considerations 🎨

Testing 🧪

How was this tested 👻

• Manual testing
• E2E tests added
• Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

pnpm test src/lib/utilities/route-for-base-path.test.ts

Run the following ⬇️

cd server
docker build -t temporal-ui-local .
docker run -p 5252:8080 -e TEMPORAL_UI_PUBLIC_PATH=/custom-path -e TEMPORAL_ADDRESS=host.docker.internal:7233 temporal-ui-local

pnpm build:server 
pnpm dev:ui-server

Then go to http://localhost:5252/custom-path/namespaces/default and navigate to different links (e.g. /workflows).

• Verify /custom-path does not appear twice in the URL and links work as expected

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Related Slack thread.

Docs

Any docs updates needed?